Skip to content

Promote -c flag#4619

Open
jlee303 wants to merge 4 commits intofacebook:devfrom
jlee303:display_option_c
Open

Promote -c flag#4619
jlee303 wants to merge 4 commits intofacebook:devfrom
jlee303:display_option_c

Conversation

@jlee303
Copy link

@jlee303 jlee303 commented Mar 9, 2026

Promote -c to be standard option displayed by -h #4455
Thanks for the suggestion @tats-u !

@meta-cla meta-cla bot added the CLA Signed label Mar 9, 2026
@Cyan4973
Copy link
Contributor

While moving -c/--stdout in a more visible place is straightforward and correctly done in this PR,

the situation is less clear for -q/--quiet.
I think it makes more sense to keep -q alongside -v, as they both impact the same thing (i.e. the amount of verbosity). Other -h help screens agree with this principle, with the exception of those which follow the alphabetical order strictly.

So we could also bring -v along with -q? But now it introduces another question: which options deserves to be tagged "important", and be promoted to first help screen (as opposed to the more advanced and complete -H). We can't just promote everything blindly, because if everything is important, nothing is.

In order to keep this PR short, I would separate the 2 issues, so that -c/--stdout can be validated and merged without delay nor uncertainty related to other less obvious topics.

@jlee303 jlee303 changed the title Promote -c and -q flag Promote -c flag Mar 12, 2026
@jlee303
Copy link
Author

jlee303 commented Mar 12, 2026

While moving -c/--stdout in a more visible place is straightforward and correctly done in this PR,

the situation is less clear for -q/--quiet. I think it makes more sense to keep -q alongside -v, as they both impact the same thing (i.e. the amount of verbosity). Other -h help screens agree with this principle, with the exception of those which follow the alphabetical order strictly.

So we could also bring -v along with -q? But now it introduces another question: which options deserves to be tagged "important", and be promoted to first help screen (as opposed to the more advanced and complete -H). We can't just promote everything blindly, because if everything is important, nothing is.

In order to keep this PR short, I would separate the 2 issues, so that -c/--stdout can be validated and merged without delay nor uncertainty related to other less obvious topics.

Makes sense! I've updated pr and title to only promote -c. Let me know if there's anything else :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants